Harden CommandTree testing, summaries, execution, and SEO#19
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TLDR
Adds fail-fast CommandTree test execution while broadening command execution, AI summary, coverage, and website SEO hardening across the branch.
Details
What Was Added
src/summaryOrchestration.tsandsrc/state.ts.text/uri-list, plain text paths, and CommandTree command IDs.-ffile path.What Was Changed/Deleted
make testnow runs fail-fast by using Mocha--bailfor unit tests andvscode-test --bailfor VS Code-hosted tests, while excluding@exclude-ciauth-gated tests from the main test target.test-exclude-ciis now an alias fortest, keeping existing workflows on the same fail-fast path.Spec/Doc Changes
Agents.md,Claude.md, and.github/PULL_REQUEST_TEMPLATE.md..claude/skills/website-audit/SKILL.mdand.codex/skills/*links for local skill access.Breaking Changes
No extension runtime breaking changes. The intentional workflow change is that
make testis now fail-fast and excludes@exclude-citests by default; auth-gated Copilot tests remain outside the default CI path.How Do The Automated Tests Prove It Works?
make cipassed in this session after the branch changes:118 passing.372 passing.93.11%statements/lines,88.33%branches, and94.87%functions, then ratchetedcoverage-thresholds.json.Specific tests added or updated in the diff prove the main behaviors:
automatic selection picks a concrete model without prompting,activate() returns while initial discovery is still in flight, copy-path E2E tests, Docker execution E2E tests, make-executable E2E tests, script drag E2E tests, sort-order E2E tests, summary tooltip E2E tests, undefined-argument no-op E2E tests, markdown description unit tests, task helper unit tests, and DB lock recovery unit tests.